app/src/main/res/drawable/ic_check_circle.xml (1)

3-3: 경로 데이터의 중복 명령어 정리 제안

pathData의 끝부분에 Q480,480 480,480Q480,480 480,480... 와 같이 동일한 좌표를 반복하는 이차 베지에(quadratic Bezier) 명령어들이 있습니다. 이 명령어들은 시각적으로 기여하지 않으며 불필요한 코드로 보입니다. 파일을 정리하기 위해 이 부분을 제거해도 아이콘 모양에는 영향이 없습니다.

다음과 같이 정리할 수 있습니다:

-    <path android:fillColor="@android:color/white" android:pathData="M424,664L706,382L650,326L424,552L310,438L254,494L424,664ZM480,880Q397,880 324,848.5Q251,817 197,763Q143,709 111.5,636Q80,563 80,480Q80,397 111.5,324Q143,251 197,197Q251,143 324,111.5Q397,80 480,80Q563,80 636,111.5Q709,143 763,197Q817,251 848.5,324Q880,397 880,480Q880,563 848.5,636Q817,709 763,763Q709,817 636,848.5Q563,880 480,880ZM480,800Q614,800 707,707Q800,614 800,480Q800,346 707,253Q614,160 480,160Q346,160 253,253Q160,346 160,480Q160,614 253,707Q346,800 480,800Z"/>
+    <path android:fillColor="@android:color/white" android:pathData="M424,664L706,382L650,326L424,552L310,438L254,494L424,664ZM480,880Q397,880 324,848.5Q251,817 197,763Q143,709 111.5,636Q80,563 80,480Q80,397 111.5,324Q143,251 197,197Q251,143 324,111.5Q397,80 480,80Q563,80 636,111.5Q709,143 763,197Q817,251 848.5,324Q880,397 880,480Q880,563 848.5,636Q817,709 763,763Q709,817 636,848.5Q563,880 480,880ZM480,800Q614,800 707,707Q800,614 800,480Q800,346 707,253Q614,160 480,160Q346,160 253,253Q160,346 160,480Q160,614 253,707Q346,800 480,800Z"/>

app/src/main/java/com/sampoom/android/feature/part/ui/PartScreen.kt (1)

119-119: 그룹 섹션과의 일관성 검토 필요

카테고리 섹션의 ErrorContent와 EmptyContent에 .fillMaxWidth()가 추가되었지만, 그룹 섹션(214, 221번 라인)의 동일한 컴포넌트에는 적용되지 않았습니다. UI 일관성을 위해 그룹 섹션에도 동일한 수정을 적용하는 것이 좋습니다.

그룹 섹션에도 동일한 수정을 적용하려면 다음을 추가하세요:

214번 라인:

modifier = Modifier.height(200.dp).fillMaxWidth()

221번 라인:

modifier = Modifier.height(200.dp).fillMaxWidth()

Also applies to: 126-126

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailUiState.kt (1)

5-13: 네이밍/모델링 가독성

orderDetail가 List라면 복수 의미가 명확히 드러나지 않습니다. orders 또는 orderLines 등으로의 이름 변경을 고려해주세요. 상세 단일 객체가 필요하다면 별도 OrderDetail 도메인 모델 도입도 검토할 만합니다.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailViewModel.kt (6)

33-41: orderId 설정 흐름

API에서 orderId를 나중에 세팅(setOrderIdFromApi)하는 경우 자동 로드가 되지 않습니다. 세팅 시점에 로드까지 트리거하거나, 0L이면 이벤트를 무시하는 가드가 필요합니다.

---

34-35: 미사용 변수

agencyId는 사용되지 않습니다. 제거해 주세요.

---

58-66: 중복 이벤트 처리 단순화

LoadOrder와 RetryOrder가 동일합니다. 하나로 합치거나 RetryOrder를 제거해도 됩니다.

---

68-93: 다중 로드 경쟁 상태 가능성

빠른 연속 탭에서 이전 로드가 살아있을 수 있습니다. 최신 요청만 유효하게 하려면 Job 추적 후 cancel-and-join 패턴을 권장합니다.

예시:

• private var loadJob: Job? = null
• load 호출 시 loadJob?.cancel(); loadJob = viewModelScope.launch { … }

---

64-65: 에러 클리어 일관성

ClearError가 isProcessingError만 초기화합니다. orderDetailError도 함께 초기화하거나 별도 이벤트로 분리해 명확히 해주세요.

Also applies to: 149-151

---

91-92: 상태 전체 덤프 로깅

상태 전체를 로그로 남기면 데이터 노출/성능 이슈가 있습니다. 태그드/요약 로그 또는 디버그 빌드에서만 출력으로 제한해주세요.

Also applies to: 118-119, 145-146

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailContent.kt (1)

34-64: LazyColumn 내부의 forEach 사용을 items()로 개선하는 것을 권장합니다.

현재 forEach를 사용하여 item을 추가하고 있는데, 이는 LazyColumn의 최적화 기능(list diffing, key-based recomposition)을 활용하지 못합니다. 성능 개선을 위해 items() 함수 사용을 고려해보세요.

예시:

LazyColumn(...) {
    items(order) { singleOrder ->
        OrderInfoCard(order = singleOrder)
    }
    // 나머지도 유사하게 변경
}

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListUiEvent.kt (1)

4-5: data object 사용을 권장합니다.

Kotlin 1.7.20 이상에서는 싱글톤 이벤트에 object 대신 data object를 사용하는 것이 권장됩니다. 더 나은 toString() 구현과 일관성을 제공합니다.

다음과 같이 수정하세요:

 sealed interface OrderListUiEvent {
-    object LoadOrderList : OrderListUiEvent
-    object RetryOrderList : OrderListUiEvent
+    data object LoadOrderList : OrderListUiEvent
+    data object RetryOrderList : OrderListUiEvent
 }

app/src/main/java/com/sampoom/android/feature/order/domain/model/OrderList.kt (1)

3-11: 중복된 computed 속성을 제거하는 것을 고려하세요.

totalCount와 isEmpty는 items.size와 items.isEmpty()로 직접 계산할 수 있는 값입니다. 이러한 저장된 속성은 메모리 오버헤드를 추가하면서도 실질적인 이점이 없습니다.

다음과 같이 리팩토링을 고려하세요:

 data class OrderList(
-    val items: List<Order>,
-    val totalCount: Int = items.size,
-    val isEmpty: Boolean = items.isEmpty()
+    val items: List<Order>
 ) {
+    val totalCount: Int get() = items.size
+    val isEmpty: Boolean get() = items.isEmpty()
+
     companion object Companion {
         fun empty() = OrderList(emptyList())
     }
 }

또는 호출부에서 items.size와 items.isEmpty()를 직접 사용하도록 하여 이러한 래퍼 속성을 완전히 제거할 수도 있습니다.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderResultBottomSheet.kt (2)

63-69: 조건부 로직 개선 필요

clearSuccess()와 LoadOrder가 isProcessingCancelSuccess의 상태와 무관하게 항상 호출됩니다. 이는 불필요한 API 호출을 발생시킬 수 있습니다.

다음과 같이 수정하는 것을 권장합니다:

 LaunchedEffect(uiState.isProcessingCancelSuccess) {
     if (uiState.isProcessingCancelSuccess) {
         Toast.makeText(context, context.getString(R.string.order_detail_toast_order_cancel), Toast.LENGTH_SHORT).show()
+        viewModel.clearSuccess()
+        viewModel.onEvent(OrderDetailUiEvent.LoadOrder)
     }
-    viewModel.clearSuccess()
-    viewModel.onEvent(OrderDetailUiEvent.LoadOrder)
 }

---

46-50: API 명확성 개선 고려

함수가 List<Order>를 받지만 실제로는 첫 번째 주문만 사용합니다. 단일 주문만 처리하는 경우 order: Order 파라미터로 변경하거나, 여러 주문을 처리하는 기능을 추가하는 것을 고려하세요.

단일 주문만 처리하는 경우:

-fun OrderResultBottomSheet(
-    order: List<Order>,
-    onDismiss: () -> Unit,
-    viewModel: OrderDetailViewModel = hiltViewModel()
-) {
+fun OrderResultBottomSheet(
+    order: Order,
+    onDismiss: () -> Unit,
+    viewModel: OrderDetailViewModel = hiltViewModel()
+) {

그리고 line 58-60을 다음과 같이 수정:

-    LaunchedEffect(order) {
-        if (order.isNotEmpty()) viewModel.setOrderIdFromApi(order.first().orderId)
-    }
+    LaunchedEffect(order.orderId) {
+        viewModel.setOrderIdFromApi(order.orderId)
+    }

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListScreen.kt (1)

50-53: LaunchedEffect 의존성 검토

LaunchedEffect(errorLabel)은 errorLabel이 변경될 때마다 트리거되어 주문 목록을 다시 로드합니다. 일반적으로 에러 라벨은 변경되지 않으므로 문제가 없지만, LoadOrderList 호출은 init 블록에서 이미 수행되므로 중복일 수 있습니다.

다음과 같이 단순화할 수 있습니다:

-    LaunchedEffect(errorLabel) {
+    LaunchedEffect(Unit) {
         viewModel.bindLabel(errorLabel)
-        viewModel.onEvent(OrderListUiEvent.LoadOrderList)
     }

(init 블록에서 이미 loadOrderList()가 호출되므로)

app/src/main/java/com/sampoom/android/feature/order/domain/model/Order.kt (2)

6-6: 날짜 타입 개선을 고려하세요.

createdAt이 String으로 정의되어 있어 날짜 비교, 포맷팅, 파싱 시 일관성 문제가 발생할 수 있습니다. LocalDateTime, Instant, 또는 Long (타임스탬프) 같은 적절한 날짜 타입 사용을 권장합니다.

---

28-28: 수량 타입을 Int로 변경하는 것을 고려하세요.

quantity가 Long으로 정의되어 있지만, 일반적인 주문 수량은 Int 범위(최대 ~21억)를 넘지 않습니다. Int를 사용하면 메모리 효율성이 높아지고 불필요한 타입 변환을 줄일 수 있습니다.

app/src/main/java/com/sampoom/android/feature/order/data/repository/OrderRepositoryImpl.kt (2)

24-29: 에러 처리를 개선하세요.

dto.message만으로 Exception을 던지면 원본 에러의 맥락(에러 코드, 스택 트레이스 등)이 손실됩니다. 커스텀 예외 타입을 정의하거나 최소한 에러 코드를 포함하는 것을 권장합니다.

예시:

if (!dto.success) {
    throw OrderProcessingException(
        message = dto.message ?: "Unknown error",
        code = dto.errorCode
    )
}

---

31-35: 메서드 시맨틱과 반환 타입의 불일치를 해결하세요.

getOrderDetail이 OrderList를 반환하고 있는데, "상세 조회"는 일반적으로 단일 엔티티를 반환합니다. 백엔드가 실제로 리스트를 반환한다면 메서드명을 변경하거나, 단일 Order를 반환하도록 수정하는 것을 고려하세요.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailScreen.kt (1)

92-109: 작업 진행 중 상태를 표시하세요.

사용자가 취소 또는 입고 처리 버튼을 클릭한 후 작업이 완료될 때까지 로딩 인디케이터나 버튼 비활성화 같은 피드백이 없습니다. 이로 인해 사용자가 버튼을 여러 번 클릭할 수 있습니다.

uiState에 isProcessingCancel 또는 isProcessingReceive 같은 로딩 플래그를 추가하고, 이를 기반으로 버튼을 비활성화하는 것을 권장합니다.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

app/src/main/java/com/sampoom/android/app/navigation/AppNavHost.kt (1)

155-162: agencyId 하드코딩 재발 — 실제 사용자 컨텍스트 사용으로 교체하세요

이전 리뷰에서 지적된 동일 이슈입니다. routeOrderDetail(1, order.orderId)는 실제 agencyId를 사용해야 합니다.

- parentNavController.navigate(routeOrderDetail(1, order.orderId))
+ val agencyId = userSession.currentAgencyId
+ parentNavController.navigate(routeOrderDetail(agencyId, order.orderId))

app/src/main/java/com/sampoom/android/feature/part/ui/PartViewModel.kt (1)

93-115: 예외 처리가 올바르게 구현되었으며, 경쟁 조건 방지 로직이 적절합니다.

구현이 잘 되어있습니다:

• Line 96과 108의 stale request 체크가 카테고리 빠른 전환 시 경쟁 조건을 방지합니다
• CancellationException 처리가 올바릅니다 (lines 104-106)
• 에러 경로에서도 stale 체크를 수행하여 일관성을 유지합니다

loadCategory()와 loadGroup() 간 에러 처리 패턴이 중복됩니다. 선택적으로 공통 헬퍼 함수 추출을 고려할 수 있습니다:

private suspend fun <T> executeWithErrorHandling(
    updateLoading: (Boolean) -> Unit,
    updateError: (String?) -> Unit,
    block: suspend () -> T
): Result<T> {
    updateLoading(true)
    return try {
        val result = block()
        updateError(null)
        Result.success(result)
    } catch (ce: CancellationException) {
        throw ce
    } catch (throwable: Throwable) {
        val backendMessage = throwable.serverMessageOrNull()
        updateError(backendMessage ?: (throwable.message ?: errorLabel))
        Result.failure(throwable)
    } finally {
        updateLoading(false)
    }
}

하지만 현재 구현도 명확하고 읽기 쉬우므로 필수 변경사항은 아닙니다.

app/src/main/java/com/sampoom/android/feature/part/ui/PartListViewModel.kt (1)

78-78: 로그 문 위치를 개선하는 것을 고려해보세요.

현재 로그 문은 성공과 실패 케이스 모두에서 실행됩니다. 의도적인 것이라면 문제없지만, 다음 중 하나를 고려해보세요:

• 성공 케이스 내부로 이동하여 성공 시에만 로깅
• finally 블록으로 이동하여 의도를 명확히 표현
• 성공/실패에 따라 다른 로그 레벨 사용

성공 케이스 내부로 이동하는 예시:

             try {
                 val partList = getPartListUseCase(groupId)
                 _uiState.update {
                     it.copy(
                         partList = partList.items,
                         partListLoading = false,
                         partListError = null
                     )
                 }
+                Log.d(TAG, "submit: ${_uiState.value}")
             } catch (ce: CancellationException) {
                 throw ce
             } catch (throwable: Throwable) {
                 val backendMessage = throwable.serverMessageOrNull()
                 _uiState.update {
                     it.copy(
                         partListLoading = false,
                         partListError = backendMessage ?: (throwable.message ?: errorLabel)
                     )
                 }
             }
-            Log.d(TAG, "submit: ${_uiState.value}")
         }
     }

app/src/main/java/com/sampoom/android/feature/outbound/data/repository/OutboundRepositoryImpl.kt (2)

10-10: CancellationException 처리가 올바르게 개선되었습니다.

runCatching에서 명시적인 try/catch 블록으로 전환하여 코루틴 취소가 올바르게 전파됩니다. 이전 리뷰 피드백을 잘 반영했습니다.

다만, 이전 리뷰에서 제안한 도메인 예외 사용과 null-safe 메시지 처리는 아직 적용되지 않았습니다. dto.message가 null일 수 있으므로 다음과 같은 개선을 고려해주세요:

-if (!dto.success) throw Exception(dto.message)
+if (!dto.success) throw Exception(dto.message ?: "Operation failed")

또는 프로젝트 전체에서 사용할 도메인 예외 클래스를 정의하는 것을 권장합니다.

Also applies to: 21-31, 37-45, 49-57, 61-69, 76-84

---

26-26: 코드 스타일: catch 블록의 콜론 앞뒤 공백 불일치

catch (ce : CancellationException) 패턴에서 콜론 앞뒤에 공백이 있습니다. Kotlin 컨벤션에 따르면 콜론 앞에는 공백이 없고 뒤에만 있어야 합니다.

-} catch (ce : CancellationException) {
+} catch (ce: CancellationException) {

동일한 파일의 catch (t : Throwable) 부분도 동일하게 수정이 필요합니다.

Also applies to: 41-41, 53-53, 65-65, 80-80

app/src/main/java/com/sampoom/android/feature/cart/data/repository/CartRepositoryImpl.kt (3)

9-9: CancellationException import가 OutboundRepositoryImpl과 다릅니다.

이 파일은 kotlinx.coroutines.CancellationException을 import하지만, OutboundRepositoryImpl.kt는 kotlin.coroutines.cancellation.CancellationException을 import합니다.

두 import는 기능적으로 동일하지만 (kotlinx.coroutines 버전은 typealias), 일관성을 위해 동일한 import를 사용하는 것을 권장합니다. Kotlin 1.4+ 표준 라이브러리의 kotlin.coroutines.cancellation.CancellationException을 사용하는 것이 선호됩니다.

---

26-34: CancellationException 처리가 올바르게 구현되었습니다.

runCatching에서 명시적인 try/catch로 전환하여 코루틴 취소 시맨틱이 올바르게 유지됩니다.

하지만 OutboundRepositoryImpl과 마찬가지로, dto.message가 null일 수 있으므로 null-safe 처리를 추가하는 것을 권장합니다:

-if (!dto.success) throw Exception(dto.message)
+if (!dto.success) throw Exception(dto.message ?: "Cart operation failed")

Also applies to: 38-46, 50-58, 65-73

---

32-32: 코드 스타일: Throwable catch 블록의 콜론 공백

catch (t : Throwable) 선언에서 콜론 앞뒤의 공백이 Kotlin 컨벤션과 일치하지 않습니다. 콜론 앞에는 공백이 없어야 합니다.

-} catch (t : Throwable) {
+} catch (t: Throwable) {

참고: 동일한 파일의 catch (ce: CancellationException) 부분은 올바르게 작성되어 있습니다.

Also applies to: 44-44, 56-56, 71-71

app/src/main/java/com/sampoom/android/core/util/FormatDate.kt (2)

11-13: 오프셋 감지 휴리스틱이 취약합니다 — 파싱 시도 우선 흐름으로 리팩터 권장

contains('-').and(colonCount>=3) 방식은 -0300, +09 등 유효한 형식을 놓칠 수 있고, 반대로 우연한 문자열을 오인식할 수 있습니다. 오프셋 유무를 “추측”하지 말고 “파싱을 먼저 시도”하세요.

권장 리팩터(개요):

• 1순위: ISO_OFFSET_DATE_TIME로 파싱 → 기기 타임존 변환 → LocalDate
• 실패 시: yyyy-MM-dd'T'HH:mm:ss[.fraction(0..6)]
• 그래도 실패 시: ISO_LOCAL_DATE

예시:

-        val hasOffset =
-            dateString.endsWith("Z") || dateString.contains('+') || dateString.contains('-')
-                .and(dateString.count { it == ':' } >= 3)
-        val date = if (hasOffset) {
-            val inFmt = java.time.format.DateTimeFormatter.ISO_OFFSET_DATE_TIME
-            java.time.OffsetDateTime.parse(dateString, inFmt).toLocalDate()
-        } else {
-            val inFmt = java.time.format.DateTimeFormatterBuilder()
-                .appendPattern("yyyy-MM-dd'T'HH:mm:ss")
-                .optionalStart()
-                .appendFraction(java.time.temporal.ChronoField.NANO_OF_SECOND, 0, 6, true)
-                .optionalEnd()
-                .toFormatter(java.util.Locale.ROOT)
-            java.time.LocalDateTime.parse(dateString, inFmt).toLocalDate()
-        }
+        val date = runCatching {
+            java.time.OffsetDateTime.parse(
+                dateString,
+                java.time.format.DateTimeFormatter.ISO_OFFSET_DATE_TIME
+            ).atZoneSameInstant(java.time.ZoneId.systemDefault()).toLocalDate()
+        }.recoverCatching {
+            val inFmt = java.time.format.DateTimeFormatterBuilder()
+                .appendPattern("yyyy-MM-dd'T'HH:mm:ss")
+                .optionalStart()
+                .appendFraction(java.time.temporal.ChronoField.NANO_OF_SECOND, 0, 6, true)
+                .optionalEnd()
+                .toFormatter(java.util.Locale.ROOT)
+            java.time.LocalDateTime.parse(dateString, inFmt).toLocalDate()
+        }.recoverCatching {
+            java.time.LocalDate.parse(
+                dateString,
+                java.time.format.DateTimeFormatter.ISO_LOCAL_DATE
+            )
+        }.getOrThrow()

---

18-24: Formatter 매번 생성 비용을 줄이기 위해 캐싱 권장

DateTimeFormatterBuilder 결과를 매 호출마다 생성하지 말고 파일 상수로 재사용하면 가비지와 비용을 줄일 수 있습니다.

추가 코드(파일 상단에 배치):

private val FMT_LOCAL_DATE: java.time.format.DateTimeFormatter =
    java.time.format.DateTimeFormatter.ISO_LOCAL_DATE

private val FMT_LOCAL_DATE_TIME_OPT_6: java.time.format.DateTimeFormatter =
    java.time.format.DateTimeFormatterBuilder()
        .appendPattern("yyyy-MM-dd'T'HH:mm:ss")
        .optionalStart()
        .appendFraction(java.time.temporal.ChronoField.NANO_OF_SECOND, 0, 6, true)
        .optionalEnd()
        .toFormatter(java.util.Locale.ROOT)

그리고 본문에서는 FMT_LOCAL_DATE_TIME_OPT_6/FMT_LOCAL_DATE를 참조하세요.

app/src/main/java/com/sampoom/android/app/navigation/AppNavHost.kt (1)

46-48: 주문 상세 라우트 경로가 orders/.../orders/...로 중복되어 비일관적입니다

리소스 계층을 명확히 하도록 경로를 정리하세요. 권장:

• 상위 리소스(agency) → 하위 리소스(orders)

다음과 같이 변경 제안:

-const val ROUTE_ORDER_DETAIL = "orders/{agencyId}/orders/{orderId}"
-fun routeOrderDetail(agencyId: Long, orderId: Long): String = "orders/$agencyId/orders/$orderId"
+const val ROUTE_ORDER_DETAIL = "agencies/{agencyId}/orders/{orderId}"
+fun routeOrderDetail(agencyId: Long, orderId: Long): String = "agencies/$agencyId/orders/$orderId"

ROUTE_ORDER_DETAIL를 참조하는 네비게이션 그래프와 딥링크가 있다면 함께 업데이트 필요합니다.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListScreen.kt (2)

119-125: LazyColumn에 stable key 지정으로 스크롤/애니메이션 안정성 확보

아이템에 고유 ID가 있으므로 key를 지정하세요.

-                    items(uiState.orderList) { order ->
+                    items(uiState.orderList, key = { it.orderId }) { order ->
                         OrderItem(
                             order = order,
                             onClick = { onNavigateOrderDetail(order) }
                         )
                     }

---

15-16: list state 저장 방식 단순화 (선택)

rememberLazyListState()는 기본적으로 저장 가능하며 코드가 간결합니다.

-import androidx.compose.foundation.lazy.LazyListState
+import androidx.compose.foundation.lazy.rememberLazyListState
...
-    val listState = rememberSaveable(saver = LazyListState.Saver) { LazyListState() }
+    val listState = rememberLazyListState()

Also applies to: 50-50

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListViewModel.kt (1)

69-69: 로그는 Timber 등 일관 로깅으로 통일 권장

Log.d 대신 프로젝트 표준 로거 사용을 권장합니다.

-            Log.d(TAG, "submit: ${_uiState.value}")
+            Timber.d("OrderList state: %s", _uiState.value)

(해당 라이브러리를 이미 사용 중인 경우에 한함)

app/src/main/res/values/strings.xml (2)

81-96: 용어/문구 통일성 개선 제안 (선택)

한국어 자연스러움과 UI 일관성을 위해 다음을 권장합니다.

-    <string name="order_title">주문관리</string>
-    <string name="order_empty_list">주문관리 목록이 없습니다.</string>
-    <string name="order_detail_title">주문정보</string>
-    <string name="order_detail_dialog_order_cancel">주문 취소처리하시겠습니까?</string>
-    <string name="order_detail_toast_order_cancel">주문 취소처리되었습니다</string>
-    <string name="order_detail_order_receive">입고처리</string>
-    <string name="order_detail_dialog_order_receive">입고 처리하시겠습니까?</string>
-    <string name="order_detail_toast_order_receive">입고 처리되었습니다</string>
+    <string name="order_title">주문 관리</string>
+    <string name="order_empty_list">주문 목록이 없습니다.</string>
+    <string name="order_detail_title">주문 정보</string>
+    <string name="order_detail_dialog_order_cancel">주문을 취소하시겠습니까?</string>
+    <string name="order_detail_toast_order_cancel">주문이 취소되었습니다</string>
+    <string name="order_detail_order_receive">입고 처리</string>
+    <string name="order_detail_dialog_order_receive">입고를 처리하시겠습니까?</string>
+    <string name="order_detail_toast_order_receive">입고가 처리되었습니다</string>

---

111-111: 리소스 키와 값 불일치

common_slash인데 값이 -(대시)입니다. 혼동을 줄이려면 값 또는 키를 일치시키세요.

-    <string name="common_slash">-</string>
+    <string name="common_slash">/</string>

(이미 여러 곳에서 대시를 의도했다면 키를 common_dash로 새로 추가 후 점진 교체를 검토하세요.)

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListScreen.kt (2)

136-136: 불필요한 람다 래핑 제거 가능

onClick = { onClick() }는 불필요한 람다 래핑입니다. 직접 참조를 전달할 수 있습니다.

-        onClick = { onClick() },
+        onClick = onClick,

---

135-176: 접근성 개선 고려사항

OrderItem에 의미있는 semantics 정보를 추가하면 스크린 리더 사용자 경험이 향상됩니다. 예를 들어, 각 주문 카드에 대한 설명적인 contentDescription을 추가할 수 있습니다.

Card(
    onClick = onClick,
    modifier = Modifier
        .fillMaxWidth()
        .semantics {
            contentDescription = "${buildOrderTitle(order)}, ${order.agencyName}, ${order.status}"
        },
    colors = CardDefaults.cardColors(containerColor = backgroundCardColor())
) {
    // ... 기존 내용
}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailScreen.kt (2)

52-59: LaunchedEffect(취소 성공) 수정 반영 확인

이전 지적 사항이 반영되어, 성공일 때만 clearSuccess와 재로딩이 트리거됩니다. 좋습니다.

---

61-68: LaunchedEffect(입고 성공) 수정 반영 확인

위와 동일하게 조건부로 처리되어 부작용이 제거되었습니다.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderDetailScreen.kt (1)

120-147: 로딩/에러/빈 화면 높이 고정(200dp) 대신 전면 센터링 권장

부분 높이 고정은 큰 화면에서 어색합니다. 전면 채우고 중앙 정렬이 더 자연스럽습니다.

@@
-                Box(
-                    modifier = Modifier
-                        .fillMaxWidth()
-                        .height(200.dp),
+                Box(
+                    modifier = Modifier
+                        .fillMaxSize(),
                     contentAlignment = Alignment.Center
                 ) {
                     CircularProgressIndicator()
                 }
@@
-                ErrorContent(
-                    onRetry = { viewModel.onEvent(OrderDetailUiEvent.RetryOrder) },
-                    modifier = Modifier
-                        .height(200.dp)
-                        .fillMaxWidth()
-                )
+                ErrorContent(
+                    onRetry = { viewModel.onEvent(OrderDetailUiEvent.RetryOrder) },
+                    modifier = Modifier.fillMaxSize()
+                )
@@
-                EmptyContent(
-                    message = stringResource(R.string.order_empty_list),
-                    modifier = Modifier
-                        .height(200.dp)
-                        .fillMaxWidth()
-                )
+                EmptyContent(
+                    message = stringResource(R.string.order_empty_list),
+                    modifier = Modifier.fillMaxSize()
+                )

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

app/src/main/java/com/sampoom/android/feature/order/ui/OrderResultBottomSheet.kt (1)

60-62: LaunchedEffect 키 안정성 개선 확인

order.firstOrNull()?.orderId를 키로 사용한 점 좋습니다. 이전 불안정 키 문제를 적절히 해결했습니다.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListViewModel.kt (1)

51-78: 이전 리뷰 피드백이 반영되었습니다

Job 관리(loadJob)와 I/O 디스패처(withContext(Dispatchers.IO))가 적절히 추가되어 경쟁 상태와 메인 스레드 블로킹 문제가 해결되었습니다.

선택적 개선: Line 76의 로그가 오류 발생 시에도 실행됩니다. 필요시 try 블록 내부로 이동하는 것을 고려해보세요.

app/src/main/java/com/sampoom/android/feature/part/ui/PartListScreen.kt (2)

147-156: LazyColumn 항목에 key 지정으로 스크롤/상태 안정성 확보

항목 키가 없어 재구성 시 스크롤 점프/상태 손실 위험이 있습니다. 고유한 코드로 key를 부여하세요.

-                        items(uiState.partList) { part ->
+                        items(uiState.partList, key = { it.code }) { part ->
                             PartListItemCard(
                                 part = part,
                                 onClick = {

---

50-50: 잘못된/불필요한 import 제거

OutboundListUiEvent는 사용되지 않으며, 파트 화면에는 부적합한 import입니다. 제거해 주세요.

-import com.sampoom.android.feature.outbound.ui.OutboundListUiEvent

app/src/main/java/com/sampoom/android/feature/outbound/ui/OutboundListScreen.kt (3)

52-52: 불필요한 import 제거

OrderListUiEvent는 본 파일에서 사용되지 않습니다. 제거하여 경고를 줄이세요.

-import com.sampoom.android.feature.order.ui.OrderListUiEvent

---

171-176: Spacer 대신 LazyColumn contentPadding 사용

하단 여유 공간을 item { Spacer(...) }로 넣기보다 contentPadding을 사용하면 성능과 가독성이 좋아집니다.

-                        LazyColumn(
+                        LazyColumn(
                             modifier = Modifier
                                 .fillMaxSize()
                                 .padding(horizontal = 16.dp),
-                            verticalArrangement = Arrangement.spacedBy(16.dp)
+                            verticalArrangement = Arrangement.spacedBy(16.dp),
+                            contentPadding = PaddingValues(bottom = 100.dp)
                         ) {
@@
-                            item { Spacer(Modifier.height(100.dp)) }

Also applies to: 191-191

---

177-189: Lazy 항목에 안정적 key 부여

item { ... }에 key가 없어 재구성 안정성이 떨어집니다. 카테고리/그룹 조합으로 key를 부여하세요.

-                            uiState.outboundList.forEach { category ->
-                                category.groups.forEach { group ->
-                                    item {
+                            uiState.outboundList.forEach { category ->
+                                category.groups.forEach { group ->
+                                    item(key = "${category.categoryName}:${group.groupName}") {
                                         OutboundSection(
                                             categoryName = category.categoryName,
                                             groupName = group.groupName,

app/src/main/java/com/sampoom/android/feature/order/ui/OrderResultBottomSheet.kt (1)

3-3: 잘못된 플랫폼 import 제거

import android.R.attr.onClick는 Compose 코드에 불필요하며 오해를 유발합니다. 삭제하세요.

-import android.R.attr.onClick

app/src/main/java/com/sampoom/android/feature/cart/ui/CartListScreen.kt (3)

50-52: 불필요한 import 정리

아래 import는 사용되지 않습니다. 제거하여 경고를 줄이세요.

-import com.sampoom.android.feature.order.ui.OrderListUiEvent
-import kotlin.collections.forEach

---

167-176: Spacer 대신 LazyColumn contentPadding 사용

하단 여백을 contentPadding으로 처리해 리스트 성능과 가독성을 개선하세요.

-                        LazyColumn(
+                        LazyColumn(
                             modifier = Modifier
                                 .fillMaxSize()
                                 .padding(horizontal = 16.dp),
-                            verticalArrangement = Arrangement.spacedBy(16.dp)
+                            verticalArrangement = Arrangement.spacedBy(16.dp),
+                            contentPadding = PaddingValues(bottom = 100.dp)
                         ) {
@@
-                            item { Spacer(Modifier.height(100.dp)) }

Also applies to: 186-186

---

172-186: Lazy 항목에 안정적 key 부여

카테고리/그룹 아이템에 key가 없어 상태/스크롤 안정성이 떨어집니다. 조합 키를 부여하세요.

-                            uiState.cartList.forEach { category ->
-                                category.groups.forEach { group ->
-                                    item {
+                            uiState.cartList.forEach { category ->
+                                category.groups.forEach { group ->
+                                    item(key = "${category.categoryName}:${group.groupName}") {
                                         CartSection(

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListViewModel.kt (1)

38-40: 초기 로드 중복 호출 문제 지속

이전 리뷰에서 지적된 것처럼, ViewModel의 init 블록과 UI의 LaunchedEffect가 모두 loadOrderList()를 호출하여 중복 로드가 발생합니다. init 블록을 제거하고 UI에서만 이벤트를 통해 로드를 트리거하도록 일원화하세요.

app/src/main/java/com/sampoom/android/feature/order/ui/OrderListViewModel.kt (2)

70-70: 코드 스타일 일관성 개선

닫는 괄호 앞에 불필요한 공백이 있습니다.

-                        orderError = backendMessage ?: (throwable.message ?: errorLabel )
+                        orderError = backendMessage ?: (throwable.message ?: errorLabel)

---

74-74: 프로덕션 로그 검토 권장

디버그 로그가 프로덕션 빌드에도 남아있을 수 있습니다. 팀의 로깅 정책에 따라 제거하거나 조건부로 활성화하는 것을 고려하세요.

app/src/main/java/com/sampoom/android/feature/part/ui/PartListScreen.kt (2)

50-50: 사용하지 않는 import 제거 권장

OutboundListUiEvent를 파일 내에서 사용하지 않습니다. 불필요한 import는 제거하는 것이 좋습니다.

-import com.sampoom.android.feature.outbound.ui.OutboundListUiEvent

---

103-110: 로딩 UX 패턴 개선 권장

현재 uiState.partListLoading이 true일 때 pull-to-refresh 인디케이터와 전체 화면 CircularProgressIndicator가 동시에 표시됩니다. 일반적인 pull-to-refresh UX 패턴은 다음과 같습니다:

• 초기 로딩: 중앙에 로딩 인디케이터 표시
• 새로고침: 기존 리스트를 유지하고 상단의 pull 인디케이터만 표시

현재 구현은 새로고침 중에도 리스트를 숨기고 전체 화면 로딩을 표시하므로, 초기 로딩과 새로고침을 구분하는 상태를 추가하여 UX를 개선하는 것을 권장합니다.

app/src/main/java/com/sampoom/android/core/util/FormatDate.kt (1)

8-29: 엣지 케이스에 대한 단위 테스트 추가를 권장합니다

다양한 입력 형식(오프셋 유무, 'T' 누락, 잘못된 형식 등)에 대한 테스트를 추가하면 함수의 견고성을 검증할 수 있습니다.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro